Skip to content

review-checklist: show non-CODEOWNER reviewer when no area owner is assigned#6446

Merged
brtnfld merged 2 commits into
HDFGroup:developfrom
brtnfld:fix/review-checklist-non-owner-fallback
Jun 12, 2026
Merged

review-checklist: show non-CODEOWNER reviewer when no area owner is assigned#6446
brtnfld merged 2 commits into
HDFGroup:developfrom
brtnfld:fix/review-checklist-non-owner-fallback

Conversation

@brtnfld

@brtnfld brtnfld commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • If the load-balanced CODEOWNER reviewer is removed and a non-CODEOWNER is manually added in their place, the checklist previously showed no mention for that area and their approval did not check the box
  • For any area with no CODEOWNER in the requested reviewer set, the checklist now falls back to non-CODEOWNER reviewers (anyone assigned who is not an owner of any touched area)
  • Their @mention appears in the checklist row and their approval counts as sign-off for that area

If the only assigned CODEOWNER is removed and a non-CODEOWNER is manually
added in their place, that person previously got no checklist mention and
their approval did not check the box.

For any area with no CODEOWNER in the requested set, fall back to
non-CODEOWNER reviewers (anyone assigned who is not an owner of any
touched area). They are shown in the mention and their approval counts
as sign-off for that area.
@github-actions

github-actions Bot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Review Checklist

This PR touches the following areas. Each needs at least one
sign-off from its listed owners before merging — an approval
covering only one area does not satisfy the others.

✅ All areas have been signed off.

Adding or removing a reviewer did not trigger a workflow run, leaving
the checklist comment stale until the next push.

Add review_requested and review_request_removed to the pull_request_target
activity types so the checklist updates immediately when a reviewer is
manually added or removed.
@brtnfld brtnfld requested a review from jhendersonHDF June 11, 2026 22:24
@brtnfld brtnfld merged commit 41ea347 into HDFGroup:develop Jun 12, 2026
70 of 106 checks passed
@github-project-automation github-project-automation Bot moved this from To be triaged to Done in HDF5 - TRIAGE & TRACK Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants